name: pre-commit run_id: commands[0] env HOME: /home/jenkins env LANG: C.UTF-8 env PATH: /w/workspace/bgpcep-tox-verify-master/.tox/pre-commit/bin:/opt/pyenv/bin:/tmp/venv-jvNc/bin:/opt/pyenv/shims:/home/jenkins/.local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin:/opt/puppetlabs/bin env PIP_DISABLE_PIP_VERSION_CHECK: 1 env PYTHONHASHSEED: 3381589271 env PYTHONIOENCODING: utf-8 env SSH_AUTH_SOCK: ******************************** env TOX_ENV_DIR: /w/workspace/bgpcep-tox-verify-master/.tox/pre-commit env TOX_ENV_NAME: pre-commit env TOX_WORK_DIR: /w/workspace/bgpcep-tox-verify-master/.tox env VIRTUAL_ENV: /w/workspace/bgpcep-tox-verify-master/.tox/pre-commit env __TOX_ENVIRONMENT_VARIABLE_ORIGINAL_CI: true metadata pid: 2416 cwd: /w/workspace/bgpcep-tox-verify-master allow: /w/workspace/bgpcep-tox-verify-master/.tox/pre-commit/bin/* cmd: pre-commit run --all-files --show-diff-on-failure exit_code: 1 [WARNING] hook id `remove-tabs` uses deprecated stage names (commit) which will be removed in a future version. run: `pre-commit migrate-config` to automatically fix this. [INFO] Initializing environment for https://github.com/pre-commit/pre-commit-hooks. [WARNING] repo `https://github.com/pre-commit/pre-commit-hooks` uses deprecated stage names (commit, push) which will be removed in a future version. Hint: often `pre-commit autoupdate --repo https://github.com/pre-commit/pre-commit-hooks` will fix this. if it does not -- consider reporting an issue to that repo. [INFO] Initializing environment for https://github.com/Lucas-C/pre-commit-hooks. [INFO] Installing environment for https://github.com/pre-commit/pre-commit-hooks. [INFO] Once installed this environment will be reused. [INFO] This may take a few minutes... [INFO] Installing environment for https://github.com/Lucas-C/pre-commit-hooks. [INFO] Once installed this environment will be reused. [INFO] This may take a few minutes... trim trailing whitespace.................................................Failed - hook id: trailing-whitespace - exit code: 1 - files were modified by this hook Fixing tests/lib/pcep.py Fixing tests/test_suites/bgp_throughpcep/test_cases.py Tabs remover.............................................................Passed pre-commit hook(s) made changes. If you are seeing this message in CI, reproduce locally with: `pre-commit run --all-files`. To run `pre-commit` as part of git workflow, use `pre-commit install`. All changes made by hooks: diff --git a/tests/lib/pcep.py b/tests/lib/pcep.py index 7626b404cd..1b303cc019 100644 --- a/tests/lib/pcep.py +++ b/tests/lib/pcep.py @@ -60,7 +60,7 @@ def get_pcep_topology_hop_count(hop): gc.collect() return topology_count - + def start_pcc_mock( pcc: int, @@ -70,7 +70,7 @@ def start_pcc_mock( ) -> subprocess.Popen: """Starts pcep-pcc-mock.jar tool as PCC device simulator. - This test tool is used for simulating PCC enabled device. + This test tool is used for simulating PCC enabled device. TODO: ADD DOCUMENTATION Args: @@ -116,7 +116,7 @@ def start_pcc_mock_with_flapping( {lsp} \ {log_file_name} \ {interval}" - + process = infra.shell(start_mock_script_command, run_in_background=True) return process diff --git a/tests/test_suites/bgp_throughpcep/test_cases.py b/tests/test_suites/bgp_throughpcep/test_cases.py index fdb7ebbf19..45c7e94e03 100644 --- a/tests/test_suites/bgp_throughpcep/test_cases.py +++ b/tests/test_suites/bgp_throughpcep/test_cases.py @@ -131,7 +131,7 @@ class TestBgpAppManyPeerPrefixCount: self.iteration += 1 log.warn(f"{self.iteration}") return f"{self.iteration}.{self.iteration}.{self.iteration}.{self.iteration}/32" - + def reset_hop_generator(self): self.iteration = 0 @@ -159,7 +159,7 @@ class TestBgpAppManyPeerPrefixCount: with allure_step_with_separate_logging("Verify_1"): """Verify that within timeout, the correct number of new hops is in PCEP topology.""" utils.wait_until_function_returns_value(120, 1, TOTAL_LSPS, pcep.get_pcep_topology_hop_count, hop) - + with allure_step_with_separate_logging("Updater_2"): """Run updater tool to change hops, using 1 blocking http thread.""" hop = self.get_next_hop() @@ -224,7 +224,7 @@ class TestBgpAppManyPeerPrefixCount: with allure_step_with_separate_logging("Verify_6"): """Verify that within timeout, the correct number of new hops is in PCEP topology.""" utils.wait_until_function_returns_value(120, 1, TOTAL_LSPS, pcep.get_pcep_topology_hop_count, hop) - + with allure_step_with_separate_logging("Updater_7"): """Run updater tool to change hops, using 1 blocking http thread.""" hop = self.get_next_hop()